implemented a local Player (as spotify is kind of blocked) by bmerkle · Pull Request #1888 · microsoft/TypeAgent

bmerkle · 2026-02-03T23:44:38Z

implemented a local Player
there are currently problems with spotify to get a developer API key and application, hence here is a local Player basic implementation as alternative

…n (based on V2 behaviour) - enhanced calendar client login with better authentication and error handling - fixed some arguments (e.g. passing of dates/search strings(

- there are currently problems with spotify to get a developer API key and application, hence here is a local Player basic implementation as alternative

Copilot

Pull request overview

This pull request implements a local music player agent as an alternative to Spotify, which the PR description indicates has become difficult to access for developer API keys. The implementation provides basic music playback functionality using platform-native audio players (PowerShell/Windows Media Player on Windows, afplay on macOS, mpv on Linux).

Changes:

Added new playerLocal agent package with playback controls, queue management, and file browsing capabilities
Integrated the new agent into the defaultAgentProvider configuration
Unrelated: Substantially rewrote the calendar agent to integrate with Microsoft Graph API (not mentioned in PR description)

Reviewed changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 19 comments.

Show a summary per file

File	Description
ts/pnpm-lock.yaml	Added dependencies for the new playerLocal package including play-sound, chalk, debug, and dotenv
ts/packages/defaultAgentProvider/package.json	Added music-local workspace dependency
ts/packages/defaultAgentProvider/data/config.json	Registered localPlayer agent with name "music-local"
ts/packages/agents/playerLocal/package.json	Package definition for the new local player agent
ts/packages/agents/playerLocal/src/tsconfig.json	TypeScript configuration for the playerLocal package
ts/packages/agents/playerLocal/src/localPlayerService.ts	Core service implementing local audio file playback using platform-specific commands
ts/packages/agents/playerLocal/src/agent/localPlayerSchema.ts	TypeScript type definitions for player actions
ts/packages/agents/playerLocal/src/agent/localPlayerSchema.json	Schema metadata configuration
ts/packages/agents/playerLocal/src/agent/localPlayerSchema.agr	Grammar rules for natural language command parsing (limited coverage)
ts/packages/agents/playerLocal/src/agent/localPlayerManifest.json	Agent manifest with emoji and description
ts/packages/agents/playerLocal/src/agent/localPlayerHandlers.ts	Action handlers for executing player commands
ts/packages/agents/playerLocal/src/agent/localPlayerCommands.ts	Command-line interface handlers for the agent
ts/packages/agents/playerLocal/README.md	Documentation for setup and usage
ts/packages/agents/calendar/src/calendarActionHandlerV3.ts	Complete rewrite from stub to Graph API integration (unrelated to PR)
ts/packages/agents/agentUtils/graphUtils/src/graphClient.ts	Authentication handling improvements for Graph API (unrelated to PR)
ts/packages/agents/agentUtils/graphUtils/src/calendarClient.ts	Silent login support for calendar client (unrelated to PR)

Files not reviewed (1)

ts/pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

ts/packages/agents/playerLocal/src/localPlayerService.ts

ts/packages/agents/playerLocal/package.json

ts/packages/agents/playerLocal/README.md

ts/packages/agents/playerLocal/src/localPlayerService.ts

ts/packages/agents/playerLocal/src/agent/localPlayerSchema.agr

ts/packages/agents/playerLocal/src/localPlayerService.ts

robgruen · 2026-02-05T21:14:37Z

The calendar PR is in which is now causing some conflicts. Can you please resolve them and then I will review this PR.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

- layout fixes - build, prettier, test run fine now

…stead of async fs.promises.mkdir without await. This fixes the race condition causing "ENOENT: no such file or directory" errors when tests try to write files immediately after creating directories. grammarIntegration.spec.ts: Added describeIf and hasTestKeys imports from test-lib Wrapped the "Grammar Generation via populateCache" describe block with describeIf(..., () => hasTestKeys(), ...) to skip API-dependent tests when no API keys are configured Increased the timeout from 60 seconds to 180 seconds (3 minutes) for the API call test, as LLM calls can be slow Added test-lib to package.json devDependencies

bmerkle · 2026-02-07T20:48:27Z

@robgruen I have fixed conflicts and the code so it should work now.
I also had to fix some dependency problems and some tests were failing.
AFAIK the last two had nothing to do with my changes but i fixed them anyway in this PR.

I ran everything local (build, prettier, test) and also merged in main so it should be ok now.

Please have a look, it should be fine now

bmerkle added 2 commits January 31, 2026 01:04

- replaced empty dummy implementation of calendar with working versio…

81f3cf3

…n (based on V2 behaviour) - enhanced calendar client login with better authentication and error handling - fixed some arguments (e.g. passing of dates/search strings(

- implemented a local Player

eb73a40

- there are currently problems with spotify to get a developer API key and application, hence here is a local Player basic implementation as alternative

Copilot AI review requested due to automatic review settings February 3, 2026 23:44

bmerkle had a problem deploying to development-fork February 3, 2026 23:44 — with GitHub Actions Error

Copilot started reviewing on behalf of bmerkle February 3, 2026 23:45 View session

Copilot AI reviewed Feb 3, 2026

View reviewed changes

Update ts/packages/agents/playerLocal/src/localPlayerService.ts

c4659a3

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

bmerkle had a problem deploying to development-fork February 7, 2026 19:06 — with GitHub Actions Error

Update ts/packages/agents/playerLocal/src/localPlayerService.ts

1dd99a5

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

bmerkle had a problem deploying to development-fork February 7, 2026 19:06 — with GitHub Actions Error

Update ts/packages/agents/playerLocal/src/localPlayerService.ts

380a4a2

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

bmerkle had a problem deploying to development-fork February 7, 2026 19:07 — with GitHub Actions Error

Update ts/packages/agents/playerLocal/package.json

d8d1c24

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

bmerkle had a problem deploying to development-fork February 7, 2026 19:08 — with GitHub Actions Error

Update ts/packages/agents/playerLocal/README.md

9a0887c

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

bmerkle had a problem deploying to development-fork February 7, 2026 19:08 — with GitHub Actions Error

Merge branch 'main' into localPlayer

5cd9225

bmerkle had a problem deploying to development-fork February 7, 2026 19:14 — with GitHub Actions Error

- code fixes

1c54e01

- layout fixes - build, prettier, test run fine now

bmerkle had a problem deploying to development-fork February 7, 2026 19:38 — with GitHub Actions Error

Merge branch 'microsoft:main' into localPlayer

599a66b

bmerkle had a problem deploying to development-fork February 7, 2026 19:39 — with GitHub Actions Error

fixed dependencies

30d3236

bmerkle had a problem deploying to development-fork February 7, 2026 19:45 — with GitHub Actions Error

bmerkle had a problem deploying to development-fork February 7, 2026 20:40 — with GitHub Actions Error

format

daf06f1

bmerkle had a problem deploying to development-fork February 7, 2026 20:46 — with GitHub Actions Error

Merge branch 'main' into localPlayer

9a5d759

bmerkle requested a deployment to development-fork February 8, 2026 17:24 — with GitHub Actions Waiting

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

implemented a local Player (as spotify is kind of blocked)#1888

implemented a local Player (as spotify is kind of blocked)#1888
bmerkle wants to merge 14 commits intomicrosoft:mainfrom
bmerkle:localPlayer

bmerkle commented Feb 3, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

robgruen commented Feb 5, 2026

Uh oh!

bmerkle commented Feb 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

bmerkle commented Feb 3, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

robgruen commented Feb 5, 2026

Uh oh!

bmerkle commented Feb 7, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants